Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix total ram reporting #1161

Merged
merged 1 commit into from
Jan 16, 2024
Merged

Fix total ram reporting #1161

merged 1 commit into from
Jan 16, 2024

Conversation

rzblue
Copy link
Contributor

@rzblue rzblue commented Jan 15, 2024

Fixes #1154

@rzblue rzblue requested a review from a team as a code owner January 15, 2024 20:02
@mcm001
Copy link
Contributor

mcm001 commented Jan 15, 2024

Are we sure this order doesn't ever change?

matt@LAPTOP-F96B9E90:/mnt/c/Users/matth$ free -m
               total        used        free      shared  buff/cache   available
Mem:           15672         321       15179           2         171       15105
Swap:           4096           0        4096
matt@LAPTOP-F96B9E90:/mnt/c/Users/matth$

@BytingBulldogs3539
Copy link
Contributor

BytingBulldogs3539 commented Jan 16, 2024

Are we sure this order doesn't ever change?

matt@LAPTOP-F96B9E90:/mnt/c/Users/matth$ free -m
               total        used        free      shared  buff/cache   available
Mem:           15672         321       15179           2         171       15105
Swap:           4096           0        4096
matt@LAPTOP-F96B9E90:/mnt/c/Users/matth$

image
Order does not seem to change.

Changes looked good the command for pi and linux were different and should not be.

Tested on pi5 and looks good.

@mcm001 mcm001 merged commit f13a507 into PhotonVision:master Jan 16, 2024
23 checks passed
laviRZ added a commit to laviRZ/photonvision that referenced this pull request Jan 16, 2024
* Load libquadmath on Windows (PhotonVision#1163)

Nobody reads these, right? This probably won't make things worse. Surely.

* Add RKNN / Object Detection Pipeline (PhotonVision#1144)

Tested on Orange Pi 5 and Cool Pi 4B. Merge with parts of the OpenCV DNN PR. 

Adds support for YOLOv5s models for Rockchip CPUs with a NPU. Right now hard coded to a note model from alex_idk. Very much still incubating and largely untested.

* Add LL3 image from photon-image-modifier (PhotonVision#1166)

* LL3

* Update build.yml

* Fix total ram reporting (PhotonVision#1161)

* formatting

* spotless

* unpacked file

* update readme

* formatting of rknn-readme

---------

Co-authored-by: Matt <[email protected]>
Co-authored-by: Mohammad Durrani <[email protected]>
Co-authored-by: Programmers3539 <[email protected]>
Co-authored-by: Ryan Blue <[email protected]>
laviRZ added a commit to laviRZ/photonvision that referenced this pull request Jan 16, 2024
* Load libquadmath on Windows (PhotonVision#1163)

Nobody reads these, right? This probably won't make things worse. Surely.

* Add RKNN / Object Detection Pipeline (PhotonVision#1144)

Tested on Orange Pi 5 and Cool Pi 4B. Merge with parts of the OpenCV DNN PR. 

Adds support for YOLOv5s models for Rockchip CPUs with a NPU. Right now hard coded to a note model from alex_idk. Very much still incubating and largely untested.

* Add LL3 image from photon-image-modifier (PhotonVision#1166)

* LL3

* Update build.yml

* Fix total ram reporting (PhotonVision#1161)

* formatting

* spotless

* unpacked file

* update readme

* formatting of rknn-readme

---------

Co-authored-by: Matt <[email protected]>
Co-authored-by: Mohammad Durrani <[email protected]>
Co-authored-by: Programmers3539 <[email protected]>
Co-authored-by: Ryan Blue <[email protected]>
@rzblue rzblue deleted the fix-total-ram branch August 16, 2024 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Total ram not reported correctly
3 participants